Skip to content

Fix bug #662 - cookies not being maintained correctly over session involving redirects+set-cookie - need to save cookie domain in sessions file - #1304

Closed
barny wants to merge 1 commit into
httpie:masterfrom
barny:master

Conversation

@barny

@barny barny commented Feb 25, 2022

Copy link
Copy Markdown

Observed problems using --follow with --session, that on resuming using the saved session but expired authentication cookies stored in the session, httpie was generating duplicated cookies - saving the domain in the session file resolved this.

@barny barny changed the title Fix bug #662 - save domain in sessions file Fix bug #662 - save cookie domain in sessions file Feb 25, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #1304 (aa36b0e) into master (4d7d6b6) will decrease coverage by 0.76%.
The diff coverage is 95.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1304      +/-   ##
==========================================
- Coverage   97.28%   96.52%   -0.77%     
==========================================
  Files          67       85      +18     
  Lines        4235     6012    +1777     
==========================================
+ Hits         4120     5803    +1683     
- Misses        115      209      +94     
Impacted Files Coverage Δ
tests/test_binary.py 100.00% <ø> (ø)
httpie/compat.py 31.11% <27.90%> (-68.89%) ⬇️
tests/conftest.py 77.14% <58.33%> (-9.82%) ⬇️
httpie/output/lexers/http.py 59.09% <59.09%> (ø)
tests/test_ssl.py 91.01% <66.66%> (-3.93%) ⬇️
httpie/manager/__main__.py 82.35% <82.35%> (ø)
httpie/output/lexers/metadata.py 82.35% <82.35%> (ø)
httpie/models.py 94.23% <90.00%> (-3.14%) ⬇️
tests/test_uploads.py 97.70% <92.59%> (-2.30%) ⬇️
httpie/manager/core.py 92.85% <92.85%> (ø)
... and 71 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad613f2...aa36b0e. Read the comment docs.

@barny barny changed the title Fix bug #662 - save cookie domain in sessions file Fix bug #662 - cookies not being maintained correctly over session invcolving redirects/set-cookie - need to save cookie domain in sessions file Feb 25, 2022
@barny barny changed the title Fix bug #662 - cookies not being maintained correctly over session invcolving redirects/set-cookie - need to save cookie domain in sessions file Fix bug #662 - cookies not being maintained correctly over session involving redirects+set-cookie - need to save cookie domain in sessions file Feb 28, 2022
@isidentical

Copy link
Copy Markdown
Contributor

Thanks for the PR @barny, but we have decided to take a different approach to fully solve this problem in #1312.

@isidentical isidentical closed this Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants